Skip to content

Build swiftly release cleanup #335

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
May 2, 2025

Conversation

cmcgee1024
Copy link
Member

@cmcgee1024 cmcgee1024 commented Apr 25, 2025

Remove the copies of bits of SwiftlyCore functionality in the build-swiftly-release script:

  • runProgramEnv|Output
  • SwiftlyRelease/SwiftlyPlatform (for JSON decoding from the REST endpoint)
  • curl

Remove the curl dependency from the release script using the AsyncHTTPClient instead
to download libarchive.

Migrate the release script from URLs and FileManager to the new FilePath and FileSystem
component for improved readability, and type safety.

Model the pkgbuild command with the usual structure used for other
commands.

Provide a custom string convertible implementation for configuration
so that the configuration can be easily converted into a string for
logging and comparison purposes.

Create a runEcho() implementation for the build swiftly release
script so that commands can output their command-line to the log
for reproducibility.
@cmcgee1024
Copy link
Member Author

@swift-ci test macOS

@cmcgee1024 cmcgee1024 marked this pull request as ready for review May 2, 2025 18:51
@cmcgee1024 cmcgee1024 merged commit 74dfa77 into swiftlang:main May 2, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant